-
Notifications
You must be signed in to change notification settings - Fork 152
fix(await-async-events): improveuserEvent.setup()
detection
#1056
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(await-async-events): improveuserEvent.setup()
detection
#1056
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1056 +/- ##
==========================================
+ Coverage 96.44% 96.53% +0.08%
==========================================
Files 50 50
Lines 2704 2768 +64
Branches 1112 1153 +41
==========================================
+ Hits 2608 2672 +64
Misses 96 96 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@puglyfe LGTM in general! Can you update the branch, so I can give it a final review and merge? Thanks!
userEvent.setup()
callsuserEvent.setup()
detection
🎉 This PR is included in version 7.8.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Checks
Changes
Context
I had a little bit of familiarity with this from a previous fix, so figured I would take a shot at resolving the issue. There are a huge number of edge cases in trying to support this pattern, and I almost certainly haven't covered them all. But I specifically addressed the example in the original bug report as well as the related example that's shown in the testing-library docs.
Fixes #812